Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: Add named returns where missing (c4 #25) #43

Merged
merged 4 commits into from
Dec 12, 2021
Merged

fix: Add named returns where missing (c4 #25) #43

merged 4 commits into from
Dec 12, 2021

Conversation

deluca-mike
Copy link
Contributor

@deluca-mike deluca-mike commented Dec 12, 2021 •
edited by lucas-manuel
Loading

Description

Addresses code-423n4/2021-12-maple-findings#25

Integrations Checklist

  • Have any function signatures changed? If yes, outline below.
  • Have any features changed or been added? If yes, outline below.
  • Have any events changed or been added? If yes, outline below.
  • Has all documentation been updated?

Changelog

Function Signature Changes

Features

Events

@deluca-mike deluca-mike self-assigned this Dec 12, 2021
@shortcut-integration
Copy link

This pull request has been linked to Shortcut Story #4292: [c4] Issue #25 - Add named return to poolDelegate.

@deluca-mike deluca-mike force-pushed the sc-4292-c4-issue-25-add-named-return-to-pooldelegate branch from d08ca21 to f3f3667 Compare December 12, 2021 10:25
@deluca-mike deluca-mike force-pushed the sc-4292-c4-issue-25-add-named-return-to-pooldelegate branch from f3f3667 to b97df8a Compare December 12, 2021 10:26
Base automatically changed from sc-4290-c4-issue-23-use-hardcoded-solc-versions-for to main December 12, 2021 23:29
@lucas-manuel lucas-manuel merged commit 2ffdc24 into main Dec 12, 2021
@lucas-manuel lucas-manuel deleted the sc-4292-c4-issue-25-add-named-return-to-pooldelegate branch December 12, 2021 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lucas-manuel lucas-manuel lucas-manuel approved these changes

@JGcarv JGcarv Awaiting requested review from JGcarv

Assignees

@deluca-mike deluca-mike

Labels
Priority: Medium Status: Ready for Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@deluca-mike @lucas-manuel